-
-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade Couchbase support for Spring Boot 2.4 #14184
Conversation
These serious bugs are already fixed in 4.1.0-M1 so we used that version until 4.0.2 is released
…Repository that provides the same methodes
Solves CI error: 'dependencies.dependency.version' for com.couchbase.client:encryption:jar is missing. @ line 151, column 21
Co-authored-by: Marcelo Shima <marceloshima@gmail.com>
Co-authored-by: Marcelo Shima <marceloshima@gmail.com>
… into fix-couchbase
e2e tests pass too!
I tested this PR in JHipster Daily Builds and it fails to compile.
I'm able to reproduce it locally. It seems that Any ideas @mmonti? |
It looks like Spring Data 2020.0.5 has breaking changes in Couchbase that might be causing this. |
Cc @tchlyah who is the one who proposed Couchbase option in JHipster |
Ey!.. @mraible, a couple weeks ago @mikereiche merged my PR right before the spring-data release so the changes I added to jhipster to make it work are no longer needed. |
I'm looking into this. @mraible. @mmonti public PartTreeCouchbaseQuery(CouchbaseQueryMethod method, CouchbaseOperations operations, |
@mmonti @mraible - I've opened this issue. I will roll-back the changes to CouchbaseRepositoryQuery and ReactiveCouchbaseRepositoryQuery for that issue. |
@mmonti @mraible I've merged the fix. spring-projects/spring-data-couchbase#1093 |
Just to keep everyone in the loop, we've removed Couchbase as an option for JHipster 7. Using it with JDL should still work. We can add it back in if/when everything is working again. |
Added some juice to the mix. 🤑 |
Closing in favor of #15645. |
Fixes #13544.
Replaces #13770.
Please make sure the below checklist is followed for Pull Requests.